Skip to content

FIXED UI #37

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

FIXED UI #37

wants to merge 6 commits into from

Conversation

@palak-38
Copy link

@palak-38 palak-38 commented Jul 27, 2025 •
edited
Loading

  1. The window remains responsive even after a subprocess is called.
  2. One can exit the subprocess windows by clicking 'X' on top right corner of window
  3. Fixed the overall GUI experience.
  4. Replaced the missing face detection model with model available on official website. (Download link attached in code)
Screenshot 2025-07-27 155034 (Window still active while subprocess running) solves issue #18

palak-38 added 5 commits July 27, 2025 15:35
@palak-38
Update main.py (launch file)
191f5db 
1. Fixed the window becoming unresponsive after calling a subprocess
2. Enhanced the functionality by enabling the subprocesses to be terminated by pressing the (X) icon.
3.  Fixed inconsistencies between function names and called subprocesses.
4. Improved the overall GUI.
@palak-38
Update blinkDetect.py
34b80ae 
ADDED FUNCTIONALITY OF BEING STOPPED BY CLICKING ON 'X'
@palak-38
Update face-try.py
b327d8a
@palak-38
Update blinkDetect.py
28a22be 
1. CHANGED MODEL TO AVAILABLE MODEL N THE WEB, SINCE IT WAS MISSING FROM THE MODELS FOLDER
2. ADDED FUNCTIONALITY OF EXITING THE SUBPROCESS BY CLICKING 'X'
@palak-38
Merge branch 'main' into main
1054b4f
@thynash
Copy link
Collaborator

thynash commented Jul 29, 2025

@palak-38 currently the branch has merge conflicts which needs to be solved.

@Gagandeep-2003
Copy link
Owner

Gagandeep-2003 commented Aug 1, 2025

@palak-38 This branch is having merge conflict. Are you currently working on it or should I close this.

@palak-38
Copy link
Author

palak-38 commented Aug 1, 2025 •
edited
Loading

@Gagandeep-2003 i have tried fixing at my end pls check

@palak-38
Copy link
Author

palak-38 commented Aug 5, 2025

@Gagandeep-2003 pls check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@palak-38 @thynash @Gagandeep-2003